Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do not lookup release from package cloud if fork in circle config.yml #753

Merged
merged 2 commits into from
May 12, 2020

Conversation

guzzijones
Copy link
Contributor

@guzzijones guzzijones commented May 12, 2020

if package cloud token is not set use hard coded value of 1
fixes #752

PR from forks are not able to be run through unit testing. This is because the package cloud token is not set. Here we are not going to lookup the release number in package cloud if it is a fork.
This was stolen from https://github.com/StackStorm/st2-packages/blob/29a493e33de928e6df452a9102e80aaeadc5ec53/.circle/buildenv_st2.sh#L45-L51

if package cloud token is not set use hard coded value of 1
Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please provide a better title (instead of config.yml #753) and more detailed description for the PR.

This is useful for others to review the PR and for the history.

@arm4b arm4b added the bug label May 12, 2020
add comment about fork that was useful in finding the bug.

Co-authored-by: Eugen C. <[email protected]>
@guzzijones guzzijones changed the title config.yml Do not lookup release from package cloud if fork in circle config.yml May 12, 2020
@guzzijones
Copy link
Contributor Author

Looks like it passed.

Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks for the CircleCI build fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

st2web build from fork
2 participants